Summary

The run() function in src/disable.rs had a nested match → if → match pattern inside its per-definition loop that made the control flow harder to follow. This PR extracts the loop body into a focused helper.

What changed

New: ApplyOutcome enum

enum ApplyOutcome { Skipped, Patched, Failed }

New: apply_action() async helper
Handles one Action, prints a one-line status message, and returns the outcome — no tallying, no loop concerns. The dry-run check and the HTTP call are now a flat two-arm match instead of a nested if { ... } match { Ok match { ... } Err ... }.

run() loop simplified

// Before: ~25 lines, nested match-if-match
// After:
match apply_action(action, &client, &ado_ctx, &auth, opts.dry_run).await {
    ApplyOutcome::Skipped => skipped += 1,
    ApplyOutcome::Patched => patched += 1,
    ApplyOutcome::Failed  => failure += 1,
}

Complexity delta

Checklist

• cargo build — clean
• cargo test — all pass
• cargo clippy --all-targets --all-features — zero warnings
• No public API changes
• No behavioural changes

network:
  allowed:
    - defaults
    - "spsprodeus21.vssps.visualstudio.com"

Generated by Cyclomatic Complexity Reducer · 61 AIC · ⌖ 12.1 AIC · ⊞ 7.2K · ◷